Skip to content

Možnost vypnout Tracy Debug Bar#169

Open
jakubenglicky wants to merge 1 commit intomasterfrom
debug-bar-toggle
Open

Možnost vypnout Tracy Debug Bar#169
jakubenglicky wants to merge 1 commit intomasterfrom
debug-bar-toggle

Conversation

@jakubenglicky
Copy link
Contributor

@jakubenglicky jakubenglicky commented Sep 20, 2021

No description provided.

@jakubenglicky jakubenglicky marked this pull request as ready for review September 20, 2021 14:43
@jakubenglicky jakubenglicky self-assigned this Sep 20, 2021
MilanPala
MilanPala reviewed Sep 21, 2021
View reviewed changes
Copy link
Contributor

@MilanPala MilanPala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nette už samo v sobě má podporu pro zapínání pomocí cookies: https://doc.nette.org/cs/3.0/bootstrap#toc-vyvojarsky-vs-produkcni-rezim včetně podpory IP adres. Snažil bych se využít to. Ideálně jako getenv('DEBUG_MODE') ?: FALSE

@jakubenglicky
Copy link
Contributor Author

jakubenglicky commented Sep 23, 2021

Nette už samo v sobě má podporu pro zapínání pomocí cookies: https://doc.nette.org/cs/3.0/bootstrap#toc-vyvojarsky-vs-produkcni-rezim včetně podpory IP adres. Snažil bych se využít to. Ideálně jako getenv('DEBUG_MODE') ?: FALSE

Nakonec jsem zvolil tedy nativní fci Nette, kdy si uživatel může nastavit jak bude vypadat secret cookie, ale IP je vždy povolená ta ze které se přistupuje pomocí $_SERVER['REMOTE_ADDR']

MilanPala
MilanPala approved these changes Oct 6, 2021
View reviewed changes
Copy link
Contributor

@MilanPala MilanPala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Přidal bych tam ještě hlavičku pro proxy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@MilanPala MilanPala MilanPala approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

@jakubenglicky jakubenglicky

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jakubenglicky @MilanPala